Minimal modifications to stress tests for Linux#3548
Merged
paulmedynski merged 2 commits intomainfrom Aug 8, 2025
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR modernizes the stress tests for SqlClient to better support Linux development by replacing the old Visual Studio solution file with a minimal SLNX format and updating build configurations. The changes include updating target frameworks to .NET 9.0 and .NET Framework 4.8.1, fixing build warnings, and improving documentation.
Key Changes
- Replaced traditional .sln file with minimal .slnx format for better cross-platform compatibility
- Updated all projects to target .NET Framework 4.8.1 instead of 4.8 and maintained .NET 9.0 support
- Added centralized package management with Directory.Build.props and Directory.Packages.props files
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| StressTests.slnx | New minimal solution file replacing traditional .sln format |
| SqlClientStressTest.sln | Removed old Visual Studio solution file |
| *.csproj files | Updated target frameworks from net48 to net481 and centralized package references |
| TestBase.cs | Removed CLS compliance attributes and improved formatting |
| DataTestGroup.cs | Simplified exception handling by using bare throw statements |
| DataStressFactory.cs | Simplified exception handling by using bare throw statements |
| StressTest.config | Updated example configuration with corrected field formatting |
| Directory.Build.props | Added to prevent searching up directory tree for parent configurations |
| Directory.Packages.props | Added centralized package version management |
| Readme.md | Updated documentation with corrected examples and improved formatting |
paulmedynski
commented
Aug 7, 2025
Contributor
Author
paulmedynski
left a comment
paulmedynski
left a comment
There was a problem hiding this comment.
Adding commentary for reviewers.
src/Microsoft.Data.SqlClient/tests/StressTests/SqlClient.Stress.Framework/DataStressFactory.cs
Show resolved
Hide resolved
src/Microsoft.Data.SqlClient/tests/StressTests/SqlClient.Stress.Runner/Tests/TestBase.cs
Show resolved
Hide resolved
benrr101
approved these changes
Aug 7, 2025
Contributor
benrr101
left a comment
benrr101
left a comment
There was a problem hiding this comment.
I'm gonna make a bunch more changes to this ... but if it gets you off the ground for linux support, I guess that's fine
mdaigle
approved these changes
Aug 7, 2025
paulmedynski
added a commit
that referenced
this pull request
Aug 14, 2025
* Minimal modifications to get the stress tests building and running on Linux. * Updated sample config to avoid a dummy password.
paulmedynski
added a commit
that referenced
this pull request
Aug 28, 2025
* Moved existing stress test projects from ADO into GitHub. (#3546) (This code is effectively dead since it is not being referenced anywhere, and there are no other changes to the code. As such, we can bypass the one flaky test failure.) * Minimal modifications to stress tests for Linux (#3548) * Minimal modifications to get the stress tests building and running on Linux. * Updated sample config to avoid a dummy password. * User Story 38131: Stress Tests CI - Added config file path configuration via STRESS_CONFIG_FILE environment variable. - Added support for JSON config file. - Fixed process exit code to reflect success vs failure. - Added emission to console. - Added emission of data source. - Added EntraId password-based auth. - Added CI stage. - Added NuGet package version to MDS assembly. - Added dotnet build configuration pararmeter to CI entry points. - Added MDS package version configurability to stress tests. - Added all target frameworks that MDS supports. - Each test run now creates and drops its own database to avoid collisions. - Moved to 1ES images where possible. - Using a local SQL Server that we setup as part of CI. * User Story 38131: Stress Tests CI - Added top-level pipeline flag to enable stress tests, deafult is false until tests are reliably passing.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Directory.Build.propsandDirectory.Packages.propsthat don't allow the parent config to be used.Readme.mdto reflect changes and corrected examples.